Skip to content

docs: Consolidate docs tooling requirements#16956

Merged
lizan merged 1 commit intoenvoyproxy:mainfrom
phlax:docs-tools-requirements
Jun 18, 2021
Merged

docs: Consolidate docs tooling requirements#16956
lizan merged 1 commit intoenvoyproxy:mainfrom
phlax:docs-tools-requirements

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Jun 13, 2021

Signed-off-by: Ryan Northey ryan@synca.io

Commit Message: docs: Consolidate docs tooling requirements
Additional Description:

This is a breakout from #15229

It consolidates the 2 requirements.txt files currently in /docs/ and /tools/docs -> just /tools/docs

Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Ryan Northey <ryan@synca.io>
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #16956 was opened by phlax.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Jun 13, 2021
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Jun 13, 2021

--allow-unsafe is added to the pip-compile command as when installing sphinx with bazel (in #15229) its forcing a setuptools upgrade

@moderation
Copy link
Copy Markdown
Contributor

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Jun 14, 2021
@lizan lizan merged commit 6246f6b into envoyproxy:main Jun 18, 2021
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
Signed-off-by: Ryan Northey <ryan@synca.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants